Skip to content

refactor window-list#156

Closed
NamorNiradnug wants to merge 1 commit intoWayfireWM:masterfrom
NamorNiradnug:refactor-window-list
Closed

refactor window-list#156
NamorNiradnug wants to merge 1 commit intoWayfireWM:masterfrom
NamorNiradnug:refactor-window-list

Conversation

@NamorNiradnug
Copy link
Collaborator

Probably fixes #150

This patch refactors the calculation of WayfireToplevel's width. Instead of recalculating the width manually, it simply does label.set_ellipsize and label.set_max_width_chars.

Also there are a few minor style-ish changes suggested by clang-tidy.

@NamorNiradnug
Copy link
Collaborator Author

Known problem: the WayfireToplevel widget becomes too small when all the toplevels have a short titles.

@NamorNiradnug NamorNiradnug marked this pull request as draft July 9, 2023 22:56
Copy link
Contributor

@lcolitti lcolitti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This commit simplifies things but it doesn't fix the problem. Once there are too many windows, the box starts growing, and moving to the right any widgets that are to the right of it.

For example, to the right of the window list, I have volume, network, battery, tray, and clock widgets, and if I open too many windows, the window list starts growing and pushing those widgets off the screen.

@NamorNiradnug
Copy link
Collaborator Author

This commit simplifies things but it doesn't fix the problem. Once there are too many windows, the box starts growing, and moving to the right any widgets that are to the right of it.

For example, to the right of the window list, I have volume, network, battery, tray, and clock widgets, and if I open too many windows, the window list starts growing and pushing those widgets off the screen.

Hi
Yeah, I know there's the problem, but that's why the PR is still a draft)
Thanks for your review anyway!

@ammen99
Copy link
Member

ammen99 commented Oct 16, 2025

I believe this was updated in the gtk4 port.

@ammen99 ammen99 closed this Oct 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

High CPU usage in windowlist when it's too wide

3 participants